Skip to content

fix(transcripts): complete tools from user results - #177

Draft
Lint111 wants to merge 1 commit into
Ark0N:masterfrom
Lint111:agent/split-transcript-tool-results
Draft

fix(transcripts): complete tools from user results#177
Lint111 wants to merge 1 commit into
Ark0N:masterfrom
Lint111:agent/split-transcript-tool-results

Conversation

@Lint111

@Lint111 Lint111 commented Jul 28, 2026

Copy link
Copy Markdown

Summary

  • Recognize tool_result blocks carried by user-role transcript entries.
  • End the active tool state and emit transcript:tool_end with the correct tool name and error status.
  • Replace fixed sleeps in watcher tests with condition-based vi.waitFor assertions.

Root cause

Claude transcripts record tool requests in assistant entries but commonly record their results in user entries. The watcher only completed tools from the older result path, so Codeman could continue showing a tool as running after it had finished.

Scope

This is the transcript completion slice extracted from #173. It changes only the transcript watcher and its focused tests. It does not alter terminal rendering, hooks, or session transport.

Validation

  • npx vitest run --config config/vitest.config.ts test/transcript-watcher.test.ts: 14 passed
  • npm run build: passed
  • Commit formatting hook: passed
  • git diff --check: passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant